feat: GetBulk method for new CouchDB storage implementation #49
Conversation
Codecov Report
@@ Coverage Diff @@
## main #49 +/- ##
==========================================
+ Coverage 83.63% 83.74% +0.10%
==========================================
Files 8 8
Lines 1088 1132 +44
==========================================
+ Hits 910 948 +38
- Misses 98 101 +3
- Partials 80 83 +3
Continue to review full report at Codecov.
|
- Also includes a minor refactor for the getQueryOptions function that was added in my previous commit. Signed-off-by: Derek Trider <[email protected]>
DRK3
force-pushed
the
GetBulkNewCouchDBStorageImplementation
branch
from
fa5e6c8
to
91dd4c2
Compare
| bulkGetReferences[i].ID = key | ||
| } | ||
|
|
||
| rows, err := s.db.BulkGet(context.Background(), bulkGetReferences) |
There was a problem hiding this comment.
Reminder that contexts need to be passed-in.
There was a problem hiding this comment.
Created a followup issue earlier for it: #48
There was a problem hiding this comment.
My goal is to get the storage implementations done first so that I can update the aries-framework-go functionality to use the new storage interface. Then I'll go and make those improvements that can be done easily afterwards.
Signed-off-by: Derek Trider [email protected]